Skip to content

refactor: quality gradle plugins#11046

Merged
wmontwe merged 4 commits into
thunderbird:mainfrom
wmontwe:refactor-quality-gradle-plugins
Jun 4, 2026
Merged

refactor: quality gradle plugins#11046
wmontwe merged 4 commits into
thunderbird:mainfrom
wmontwe:refactor-quality-gradle-plugins

Conversation

@wmontwe

@wmontwe wmontwe commented May 26, 2026

Copy link
Copy Markdown
Member

This pull request introduces two new custom Gradle plugins for code quality: one for Detekt (static code analysis) and one for Spotless (code formatting). It migrates all modules to use these new plugins instead of the previous script-based approach. Additionally, applies format updates and fixes some of the detektAll findings.

This harmonizes the project setup with Thunderbird Mobile Components

@wmontwe wmontwe requested a review from a team as a code owner May 26, 2026 17:13
@wmontwe wmontwe requested a review from dani-zilla May 26, 2026 17:13
@github-actions github-actions Bot added the tb-team Tasks and features handled by project maintainers label May 26, 2026
@github-actions

github-actions Bot commented May 26, 2026

Copy link
Copy Markdown
Contributor

Validation Passed: All report and feature-flag labels are correctly set.

@wmontwe wmontwe added the report: exclude Exclude changes from user-facing reports (internal, minor, or not relevant to users). label May 26, 2026
@wmontwe wmontwe changed the title Refactor quality gradle plugins refactor: quality gradle plugins May 26, 2026
dani-zilla
dani-zilla previously approved these changes Jun 1, 2026
wmontwe added 4 commits June 4, 2026 13:15
# Conflicts:
#	feature/thundermail/internal/common/src/main/kotlin/net/thunderbird/feature/thundermail/internal/common/navigation/DefaultThundermailNavigation.kt
@wmontwe wmontwe force-pushed the refactor-quality-gradle-plugins branch from ebb378b to 5931dcb Compare June 4, 2026 11:19

@rafaeltonholo rafaeltonholo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Just a nit: I would rather prefer instead of suppressing the detekt issues having them in the baseline, but not a blocker for this PR.

@wmontwe wmontwe merged commit 80574a5 into thunderbird:main Jun 4, 2026
18 checks passed
@wmontwe wmontwe deleted the refactor-quality-gradle-plugins branch June 4, 2026 12:01
@thunderbird-botmobile thunderbird-botmobile Bot added this to the Thunderbird 21 milestone Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

report: exclude Exclude changes from user-facing reports (internal, minor, or not relevant to users). tb-team Tasks and features handled by project maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants